Skip to content

Comments

[202412] Fix test_configurable_drop_counters.py for v6 topos#1019

Open
markx-arista wants to merge 1 commit intoAzure:202412from
markx-arista:202412-fix-test-configurable-drop-counters-for-v6-topo
Open

[202412] Fix test_configurable_drop_counters.py for v6 topos#1019
markx-arista wants to merge 1 commit intoAzure:202412from
markx-arista:202412-fix-test-configurable-drop-counters-for-v6-topo

Conversation

@markx-arista
Copy link

@markx-arista markx-arista commented Feb 12, 2026

Description of PR

Summary:
Fix test_neighbor_link_down for v6 topo and remove xfail
Fix test_dip_link_local and test_sip_link_local for v6 topo, but skip them for Broadcom Asics because they don't support DIP_LINK_LOCAL/SIP_LINK_LOCAL drop reasons for IPv6 packets
Manually cherry-pick of: sonic-net/sonic-mgmt#21763

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202412
  • 202505

Approach

What is the motivation for this PR?

test_configurable_drop_counters.py failed on v6 topos

How did you do it?

Fix test cases for V6 topos, and skip test_dip_link_local/test_sip_link_local for V6 topo with Broadcom Asics

How did you verify/test it?

test_neighbor_link_down passed on V6 topos and test_dip_link_local/test_sip_link_local are skipped on V6 topos with Broadcom Asics

Any platform specific information?

Broadcom Asics don't support DIP_LINK_LOCAL/SIP_LINK_LOCAL drop reasons for IPv6 packets

Supported testbed topology if it's a new test case?

Documentation

Fix test_neighbor_link_down for v6 topo
Fix test_dip_link_local and test_sip_link_local for v6 topo, but skip them for
Broadcom Asics because they don't support DIP_LINK_LOCAL/SIP_LINK_LOCAL drop
reasons for IPv6 packets

Signed-off-by: markxiao <markxiao@arista.com>
@r12f
Copy link
Contributor

r12f commented Feb 19, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants